This will break the scaffolding afaik, if we remove the template_engine we'll not get tailwind custom scaffold templates. This is done in order to override the default rails scaffolds https://github.com/rails/tailwindcss-rails/tree/main/lib/generators/tailwindcss.

I'm 100% up for removing the template_engine configuration if you can find the way to override rails scaffolds so we have _partial.html.erb for rails 6 versions 😄 .

Do tailwind scaffolds work for you if you remove this configuration?

Thanks for the insights. I'll check properly if the scaffolds work with the configuration removed.

@dixpac turns out none of the scaffolds will include the tailwind custom styling with my change. This may be a good thing as this no longer forces it on all users.
If a user wants the custom styling that comes with the gem they can simply prepend the styling directive to the generator.
See the updated Readme.

Thanks @davidteren

I don't think this is the direction we wan't to go with, since our main idea with the tailwind generators was to override default rails generators so we don't have to provide tailwindcss:scaffold.

@dhh has the last word on this, but the initial idea was just to use bin/rails g scaffold ... and have the Tailwind styles if this engine is included.

Reference

@dixpac I think we can close this PR as in its current form it does not align with @dhh's vision of having scaffold templates that demonstrate the use of Tailwind classes.

Yup!
But, if you have an inspiration or ideas in the future on how to keep default templates and fix these edge cases.. 🙏🏼

FWIW, this breaks the haml-rails default configuration and presumably similar integrations.

I didn't expect including the tailwindcss-rails gem would switch my generated views back from .html.haml to .html.erb

Feel free to explore ways where all this can coexist. But yes, with the tailwind gem installed, scaffolds should be generated with TW styling. Maybe we can detect if another, non-default template handler has already been registered, and then warn/abort, though.